-
-
Notifications
You must be signed in to change notification settings - Fork 28.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Restructure device tracker #23862
Restructure device tracker #23862
Conversation
|
||
|
||
@callback | ||
def async_setup_scanner_platform(hass: HomeAssistantType, config: ConfigType, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is only for the legacy type, right? Should we move it to legacy?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Future PR 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After a quick read, I think this looks great!
* Restructure device tracker * Docstyle * Fix typing * Lint * Lint * Fix tests
Description:
Restructure the device tracker to make it easier to add EntityPlatform support.
This PR is a first step in an overhaul that will supersede the overhaul in #21151. The plan is to do #21151 but in smaller steps + make it that instead of overhauling everything at once, we will introduce it in a way that new platforms will use EntityPlatform, while the old ones remain using the legacy code.
Once the overhaul is complete, we will no longer accept contributions using the old style. Important platforms will be migrated to use the EntityPlatform.
Related issue (if applicable): home-assistant/architecture#151
Checklist:
tox
. Your PR cannot be merged unless tests passIf the code does not interact with devices: